-
Notifications
You must be signed in to change notification settings - Fork 183
[910] Pubsub service crashes on protocol negotiation failures #924
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
@SuchitraSwain Please fix the CI CD issues. |
@SuchitraSwain : Re-ran the CI/Cd pipeline. Please fix the issues. |
@sumanjeet0012 : Hi Sumanjeet. Since, you nicely authored the pub sub example at the beginning of year, wish if you could help @SuchitraSwain arrive at a good conclusion on the issues. |
@SuchitraSwain I have made some changes in tests and created a PR on your fork, and now the tests are passing on my local PC, Take a look at the Pubsub test fix. Please review the changes, and if everything works as expected on your end, you may go ahead and merge the PR. |
@SuchitraSwain , @sumanjeet0012 : Re-ran the CI/CD pipeline. Lets review the test results too. |
What was wrong?
Issue #910
How was it fixed?
Technical Approach
To-Do
Cute Animal Picture